wayland: make deliver_key_event return void
authorRay Strode <rstrode@redhat.com>
Tue, 1 Mar 2016 13:29:31 +0000 (08:29 -0500)
committerRay Strode <rstrode@redhat.com>
Wed, 2 Mar 2016 18:07:12 +0000 (13:07 -0500)
commit619799ba3bcc8525cc78b98c34e62a34e989f550
treed36df61a60ed3c8278c9e1ef622936e8439a7f35
parent89121e058a926280dc95d1a2530edf59e35d2814
wayland: make deliver_key_event return void

deliver_key_event is sometimes called from a timeout handler and
sometimes called directly.  We currently erroneously return TRUE
(G_SOURCE_CONTINUE) in the case where it's called directly, but to
no ill effect, since we ignore that return value. In the future,
we're going to need to call it directly in other parts of the code
where the return value would be relevant and handling TRUE, would
require adding redundant code.

Instead, this commit just changes the code to always reset the timer
manually, and never rely on glib's ability to automatically reset
the timer by returning TRUE.  This makes the code smaller, too, since
there's less special casing required.

https://bugzilla.gnome.org/show_bug.cgi?id=757942
gdk/wayland/gdkdevice-wayland.c